Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add a postgres_exporter config file #9720

Merged
merged 1 commit into from
Nov 11, 2024
Merged

Conversation

tristan957
Copy link
Member

This exporter logs an ERROR if a file called postgres_exporter.yml is not located in its current working directory. We can silence it by adding an empty config file and pointing the exporter at it.

@tristan957
Copy link
Member Author

See https://github.com/prometheus-community/postgres_exporter/blob/a324fe37bca5193a293118b940b3df7ab3a8505c/cmd/postgres_exporter/main.go#L83-L86. It looks like the message changed to a warning at some point but this check always occurs.

This exporter logs an ERROR if a file called `postgres_exporter.yml` is
not located in its current working directory. We can silence it by
adding an empty config file and pointing the exporter at it.

Signed-off-by: Tristan Partin <tristan@neon.tech>
@tristan957 tristan957 force-pushed the tristan957/postgres-exporter branch from c7ab32b to 7ad2a3a Compare November 11, 2024 17:58
compute/compute-node.Dockerfile Show resolved Hide resolved
Copy link

github-actions bot commented Nov 11, 2024

5381 tests run: 5159 passed, 0 failed, 222 skipped (full report)


Flaky tests (3)

Postgres 17

  • test_timeline_detach_ancestor_interrupted_by_deletion[False-delete_timeline]: debug-x86-64

Postgres 15

Code coverage* (full report)

  • functions: 31.7% (7869 of 24806 functions)
  • lines: 49.4% (62262 of 126045 lines)

* collected from Rust tests only


The comment gets automatically updated with the latest test results
7ad2a3a at 2024-11-11T20:46:45.517Z :recycle:

@tristan957 tristan957 merged commit 4b075db into main Nov 11, 2024
81 checks passed
@tristan957 tristan957 deleted the tristan957/postgres-exporter branch November 11, 2024 20:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants